Closed Bug 1253382 Opened 9 years ago Closed 9 years ago

Add a new test that measures how long it takes to paint the content for newly opened tabs

Categories

(Testing :: Talos, defect)

defect
Not set
normal

Tracking

(firefox47 fixed)

RESOLVED FIXED
mozilla47
Tracking Status
firefox47 --- fixed

People

(Reporter: mconley, Unassigned)

References

Details

Attachments

(3 files)

There are two cases to consider with this test: 1) The tab has been opened from the parent 2) The tab has been opened from content via a target="_blank" link, for example This was spun out of the work done in bug 1251732, as we suspect there's a slow path here for e10s that we're not currently measuring.
Blocks: 1251547, 1174770
This is necessary in order to be able to open new tabs from the initial browser window, since the initial browser window needs to have its BrowserDOMWindow attribute set when it goes remote - and if it fires before the onLoad handler in browser.js, the BrowserDOMWindow attribute that's set on the XUL window (which gets copied over to the remote browser), hasn't been set yet. Review commit: https://reviewboard.mozilla.org/r/38013/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/38013/
tabpaint measures how long it takes to open a new tab and show its content. This is a mostly done work-in-progress and isn't yet ready for review. Review commit: https://reviewboard.mozilla.org/r/38015/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/38015/
Attachment #8726458 - Attachment description: MozReview Request: Bug 1253382 - [WIP] Add a new tabpaint talos test. r?jmaher → MozReview Request: Bug 1253382 - Add a new tabpaint talos test. r?jmaher
Attachment #8726458 - Flags: review?(jmaher)
Comment on attachment 8726458 [details] MozReview Request: Bug 1253382 - Add a new tabpaint talos test. r?jmaher Review request updated; see interdiff: https://reviewboard.mozilla.org/r/38015/diff/1-2/
Comment on attachment 8726458 [details] MozReview Request: Bug 1253382 - Add a new tabpaint talos test. r?jmaher https://reviewboard.mozilla.org/r/38015/#review34679 keep in mind that we need to sign addons now, so any new addons or changes to existing ones need a signed addon. If you update the source, you can see it done for all other addons. If you look in test.py you will see references to the .xpi instead of the source. Some tips for signing addons: https://wiki.mozilla.org/EngineeringProductivity/HowTo/SignExtensions ::: testing/talos/talos/test.py:254 (Diff revision 2) > + 'browser.link.open_newwindow.restriction': 2, can you provide a brief comment about what these values of 3 and 2 mean?
Attachment #8726458 - Flags: review?(jmaher)
Comment on attachment 8726457 [details] MozReview Request: Bug 1253382 - Force the initial browser remoteness only after the browser's onLoad has fired. r?jmaher Review request updated; see interdiff: https://reviewboard.mozilla.org/r/38013/diff/1-2/
Attachment #8726457 - Flags: review?(jmaher)
Comment on attachment 8726458 [details] MozReview Request: Bug 1253382 - Add a new tabpaint talos test. r?jmaher Review request updated; see interdiff: https://reviewboard.mozilla.org/r/38015/diff/2-3/
Attachment #8726458 - Flags: review?(jmaher)
Comment on attachment 8726457 [details] MozReview Request: Bug 1253382 - Force the initial browser remoteness only after the browser's onLoad has fired. r?jmaher https://reviewboard.mozilla.org/r/38013/#review34755 thanks, this looks nice and simple.
Attachment #8726457 - Flags: review?(jmaher) → review+
Comment on attachment 8726802 [details] MozReview Request: Bug 1253382 - Sign the most recent version of TalosPowers. r?jmaher https://reviewboard.mozilla.org/r/38241/#review34759 ship it real good
Attachment #8726802 - Flags: review?(jmaher) → review+
Comment on attachment 8726458 [details] MozReview Request: Bug 1253382 - Add a new tabpaint talos test. r?jmaher https://reviewboard.mozilla.org/r/38015/#review34761 thanks for the signed addon, talos.json changes and the comments in test.py. Please add an entry here: https://wiki.mozilla.org/Buildbot/Talos/Tests
Attachment #8726458 - Flags: review?(jmaher) → review+
this is for winxp e10s (regular and pgo). I don't think it is expected, most likely the cause of the pageloader addon change: https://hg.mozilla.org/integration/mozilla-inbound/rev/64db01968f1d
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: